Skip to content

Remove terminology section - #57

Merged
fdo-secretariat merged 27 commits into
fairdo-org:devfrom
maximiliani:dev
Sep 15, 2026
Merged

fdo-secretariat merged 27 commits into
fairdo-org:devfrom
maximiliani:dev

Conversation

@maximiliani

Copy link
Copy Markdown
Contributor

This PR closes #50 and does the following:

  • Terminology section was moved into the appendix
  • Terminology definitions related to concepts already defined in the Building Blocks were removed
  • Some definition conflicts were resolved
  • Editorial fixes of typos and grammar errors
  • ReSpec was configured to support dark mode

Please check if I deleted too many terminology definitions.

@maximiliani
maximiliani requested a review from jboehm1911 July 29, 2026 16:45
@maximiliani maximiliani added the documentation Improvements or additions to documentation label Jul 29, 2026
@maximiliani maximiliani linked an issue Jul 29, 2026 that may be closed by this pull request
@github-actions

github-actions Bot commented Jul 29, 2026

Copy link
Copy Markdown

🏁 PR Preview Closed

The preview for this PR is now marked as INACTIVE.

🔗 Preview URL: https://fairdo-org.github.io/fdo-architecture-spec/preview/pr-57/

The preview will remain available but shows its inactive status.

Comment thread sections/01_fdo-building-block/01-PID.html Outdated
Comment thread sections/01_fdo-building-block/04-FDO-Profiles.html Outdated
@fdo-secretariat

Copy link
Copy Markdown
Contributor

The change needs to be discussed by the group, whether we need that section at all. The current list contains terms not used in the document. If we use the respec functions all used terms are links anyway to their definition.

@maximiliani maximiliani added the Future discussion This topic should be discussed at some point in the near future. label Aug 5, 2026
Copilot AI lite review requested due to automatic review settings August 23, 2026 18:49

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

# Conflicts:
#	sections/01_fdo-building-block/04-FDO-Profiles.html
#	sections/02_fdo-processing-block/Introduction.html
@maximiliani maximiliani reopened this Aug 25, 2026
@maximiliani maximiliani self-assigned this Aug 25, 2026
@maximiliani maximiliani changed the title Move and revise terminology section Remove terminology section Sep 12, 2026
Such specification MUST define the assignment and resolution procedures as well as other organizational
and technical requirements.
The term <a>persistence</a> is being mentioned frequently in this document, as in the realm of <a>FDOs</a> <a>persistence</a> is an important property.
In regard to <a>FDOs</a> <a>persistence</a> is defined as a promise of an initiative or service provider to maintain specifications or services for a long period of time.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think persistence of FDOs not only means that the service/specifications are long-term maintained. Also the identifier itself should be persistent. And the resource being identified by the FDO should be persistent to a certain degree (hence, only resources should be turned into FDOs that are likely around for a longer time).

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, I do not see a MUST/SHOULD/MAY in this requirement. A definition is not a requirement and should maybe go to the terminology section instead.

The <a>PID system</a> MUST support global resolution and therefore global uniqueness of the <a>PIDs</a> at a
high level of stability.
A <a>PID record</a> is an at least <a>machine readable</a> structure sent back when resolving a <a>PID</a>.
A subset of the <a>PID record</a> MAY contain the <a>FDO record</a>, refer to the <a>FDO record</a>, or refer to another <a>PID</a>.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

to be discussed

<dt><h4 class="rule-heading" id="rule-r1_8">[R1-8] <dfn>Security of <a>PID Systems</a></dfn></h4></dt>
<dd>
The <a>PID system</a> MUST offer a high degree of security that only enables the owner of a<a>FDO Record</a> to manipulate it.
It SHOULD be possible to restrict the access to the whole record or to individual key-value pairs.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

to individual attributes.

@fdo-secretariat

Copy link
Copy Markdown
Contributor

The pull request is accepted as subject to change. The pull request is to large. Parts of the changes require additional editing.

@fdo-secretariat
fdo-secretariat merged commit d37aae6 into fairdo-org:dev Sep 15, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation Future discussion This topic should be discussed at some point in the near future.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove terminology section

4 participants